Add PearsonCorrelation metric#3212
Conversation
|
@kzkadc thanks for the PR, I haven't yet reviewed it in details, just a remark on the code location which could be better to be at |
|
@vfdev-5 Is it better to create |
|
@kzkadc yes, we'll move all ignite.contrib.metrics into ignite.metrics very soon, so I'd say let's just update your PR such that the metric is in ignite.metrics but we are using |
|
Please also fix code formatting issue https://github.com/pytorch/ignite/actions/runs/8469886156/job/23234585039?pr=3212 |
Co-authored-by: vfdev <vfdev.5@gmail.com>
Head branch was pushed to by a user without write access
|
@kzkadc can you please fix the issue with the implementation using non-cpu device: |
Description: added Pearson correlation coefficient metric to
ignite.contrib.metrics.regression.Check list: